[AppSwitcher] Reimplement the separator using St.Bin

This way it can be styled using CSS.
This can help to fix bug 597362.

https://bugzilla.gnome.org/show_bug.cgi?id=598651

diff --git a/data/theme/gnome-shell.css b/data/theme/gnome-shell.css
index 42097fe8de341d81450f03e8fafaa4a0e2a6f6b3..dfabaacaecb645894a1908a1877f6e3f55b5a1df 100644 (file)
--- a/data/theme/gnome-shell.css
+++ b/data/theme/gnome-shell.css
@@ -174,3 +174,8 @@ StScrollBar StButton#vhandle:hover
     border-radius: 4px;
     background: rgba(255,255,255,0.33);
 }
+
+.switcher-list .separator {
+    width: 1px;
+    background: rgba(255,255,255,0.33);
+}

diff --git a/js/ui/altTab.js b/js/ui/altTab.js
index 62b95e327efd6f4a8397e18e675eddb60b9f15d7..570b5943e45b5e7d532df8e47ec4ffcad05c248f 100644 (file)
--- a/js/ui/altTab.js
+++ b/js/ui/altTab.js
@@ -18,8 +18,6 @@ const POPUP_ARROW_COLOR = new Clutter.Color();
 POPUP_ARROW_COLOR.from_pixel(0xffffffff);
 const TRANSPARENT_COLOR = new Clutter.Color();
 TRANSPARENT_COLOR.from_pixel(0x00000000);
-const POPUP_SEPARATOR_COLOR = new Clutter.Color();
-POPUP_SEPARATOR_COLOR.from_pixel(0x80808066);
 
 const POPUP_APPICON_SIZE = 96;
 const POPUP_LIST_SPACING = 8;
@@ -385,11 +383,7 @@ SwitcherList.prototype = {
     },
 
     addSeparator: function () {
-        // FIXME: make this work with StWidgets and CSS
-        let box = new Big.Box({ padding_top: 2, padding_bottom: 2 });
-        box.append(new Clutter.Rectangle({ width: 1,
-                                           color: POPUP_SEPARATOR_COLOR }),
-                   Big.BoxPackFlags.EXPAND);
+        let box = new St.Bin({ style_class: 'separator' })
         this._separator = box;
         this._list.add_actor(box);
     },